Skip to content

Migration tenders-finder to Tinyfish SDK #175

Open
KrishnaAgarwal7531 wants to merge 3 commits intotinyfish-io:mainfrom
KrishnaAgarwal7531:mig-tendersfinder
Open

Migration tenders-finder to Tinyfish SDK #175
KrishnaAgarwal7531 wants to merge 3 commits intotinyfish-io:mainfrom
KrishnaAgarwal7531:mig-tendersfinder

Conversation

@KrishnaAgarwal7531
Copy link
Copy Markdown
Contributor

@KrishnaAgarwal7531 KrishnaAgarwal7531 commented Apr 19, 2026

Migrates Tenders Finder to the official TinyFish SDK. /api/scrape uses client.agent.stream with typed EventType.STARTED, EventType.STREAMING_URL, EventType.PROGRESS, and EventType.COMPLETE + RunStatus.COMPLETED. No Mino refs, no Supabase, .env.example present.
Added .gitignore and README.md.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 19, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: c32b2634-5912-486e-9661-42174889f865

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@simantak-dabhade simantak-dabhade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the migration @KrishnaAgarwal7531 — the SDK usage in scrape/route.ts looks clean (client.agent.stream, proper event handling, no raw fetch). Security check passed — no leaked keys this time.

Build is broken — needs fixes before merge.

1. Delete 38 unused shadcn/ui component files (blocker)

npm run build fails because files like accordion.tsx import @radix-ui/react-accordion which isn't installed. There are 38 unused component files under src/components/ui/ that the app doesn't use. Please delete all of them — the app only needs these:

button.tsx, dialog.tsx, input.tsx, label.tsx, separator.tsx, sheet.tsx, skeleton.tsx, toast.tsx, toggle.tsx, tooltip.tsx

Delete everything else in src/components/ui/.

2. Remove unused groq-sdk dependency

groq-sdk is in package.json but never imported anywhere in src/. Please:

  • Remove "groq-sdk" from package.json
  • Remove the GROQ_API_KEY lines from .env.example
  • Remove the Groq section from README.md environment variables

(Also: Groq is no longer accepting new signups, so even if it were used we'd need to switch it.)

3. Bump Next.js to 16.x

next@15.3.6 has a known security vulnerability. Please bump to "next": "^16.2.4" (matches the rest of the cookbook).

What looks good ✓

  • SDK usage is correct: client.agent.stream() inside the request handler (not module-level)
  • No Mino references, no hardcoded secrets
  • .env.example correctly named with canonical sign-up URL
  • .gitignore properly covers .env variants
  • Supabase code fully removed
  • Old Vite artifacts cleaned up

Once the build passes (npm install && npm run build), this is good to merge. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants